Skip to content

build: use default node test runner for unit test mocks#549

Open
zimeg wants to merge 4 commits intomainfrom
zimeg-build-node-test-runner
Open

build: use default node test runner for unit test mocks#549
zimeg wants to merge 4 commits intomainfrom
zimeg-build-node-test-runner

Conversation

@zimeg
Copy link
Member

@zimeg zimeg commented Feb 4, 2026

Summary

This PR updates unit tests to use the node:test package mocks and removes the sinon and @types/sinon packages with so much appreciation 🎁

Requirements

@zimeg zimeg added this to the 2.2 milestone Feb 4, 2026
@zimeg zimeg self-assigned this Feb 4, 2026
@zimeg zimeg requested a review from a team as a code owner February 4, 2026 08:15
@zimeg zimeg added dependencies Pull requests that update a dependency file semver:patch tests labels Feb 4, 2026
@codecov
Copy link

codecov bot commented Feb 4, 2026

Codecov Report

❌ Patch coverage is 99.44828% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.54%. Comparing base (bb7fade) to head (b842ce8).

Files with missing lines Patch % Lines
test/index.spec.js 93.65% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #549      +/-   ##
==========================================
+ Coverage   95.92%   96.54%   +0.62%     
==========================================
  Files          14       14              
  Lines        2697     2868     +171     
==========================================
+ Hits         2587     2769     +182     
+ Misses        110       99      -11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@zimeg zimeg modified the milestones: 2.2, 3.x Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file semver:patch tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant